Skip to content

spm migration#58

Merged
chamodanethra merged 5 commits intomainfrom
chore/spm-migration
Feb 23, 2026
Merged

spm migration#58
chamodanethra merged 5 commits intomainfrom
chore/spm-migration

Conversation

@chamodanethra
Copy link
Owner

feat: Add Swift Package Manager support for iOS and macOS.
feat: enhance biometric type detection and improve code formatting on Android.

… Android.

- Improved `detectBiometricTypes` to use system resources and reflection for more accurate identification of face, fingerprint, and iris sensors.
- Added a `matchesLabel` utility for better matching of localized biometric strings, including support for CJK characters.
- Refactored `BiometricSignaturePlugin.kt` with consistent indentation and improved code structure.
- Replaced `FingerprintManager` with `Resources` for system-level string lookups.
- Updated error handling and callback structures to be more robust.
@chamodanethra
Copy link
Owner Author

@codex, please review

@chatgpt-codex-connector
Copy link

To use Codex here, create a Codex account and connect to github.

@chamodanethra
Copy link
Owner Author

@codex, review now.

Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 9d2c2b3fc9

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +1416 to +1417
if (buttonLabel == null || terms.isEmpty()) return false
return terms.any { term ->

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P2 Badge Fallback to MULTIPLE when biometric label detection fails

detectBiometricTypes() now relies on matchesLabel for every biometric type, but matchesLabel returns false when buttonLabel is null and the reflection block that fills buttonLabel explicitly ignores failures. In environments where BiometricManager.getStrings(...) is unavailable or OEM/localized text does not match the term lists, biometricAuthAvailable() can return canAuthenticate = true with availableBiometrics = [], which regresses previous behavior that still returned a fallback type (BiometricType.MULTIPLE) and can cause clients to mis-handle available biometrics.

Useful? React with 👍 / 👎.

…hing with a feature-backed fallback and enhancing null/blank label checks.
@chamodanethra chamodanethra merged commit daff933 into main Feb 23, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant